Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update List-HostnamesInContract.ps1 #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pablolurquilopez
Copy link

Hi, thanks for the work, but was failing (to me) using PS 7.2 till I added the @() as an array. Hope this helps.

Hi, thanks for the work, but was failing (to me) using PS 7.2 till I added the @() as an array. Hope this helps.
@stuartio
Copy link
Collaborator

Hey, thanks for the PR. Can you tell me which version of Powershell you use, and which Akamai account please? I would like to reproduce the error

@pablolurquilopez
Copy link
Author

pablolurquilopez commented Jan 19, 2023 via email

@stuartio
Copy link
Collaborator

Thanks for the info @pablolurquilopez . I am trying to narrow down the problem, but I suspect the issue is with Powershell's sometimes annoying habit of converting single-item arrays into just an object of that single item. It tends to confuse PS 5. Can you tell me the email address you use to log in to Akamai please? I can try it against your Akamai access to more easily narrow it down. Also these scripts are pretty old and I can make them way more efficient now, so will overhaul them shortly. If you want to list all hostnames in a given account, check out Get-AppSecHostnameCoverage. I think that would give you the whole list, though not which property, group and contract each hostname is associated with.

stuartio pushed a commit that referenced this pull request Jan 25, 2023
Merge in A2S/akamaipowershell from cpcodes to master

* commit '93e9c2f3a2963045b4ea5bffd3eec05aba3433a4':
  Added testing, minor fixes
  Unifying usage
  Name fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants